Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Fix esc api route URLs #3506

Merged
merged 1 commit into from
Oct 16, 2023
Merged

Fix esc api route URLs #3506

merged 1 commit into from
Oct 16, 2023

Conversation

komalali
Copy link
Member

Description

We weren't including the preview path in the esc API routes, causing them to error.

Checklist:

  • I have reviewed the style guide.
  • If blogging, I have reviewed the blogging guide.
  • I have manually confirmed that all new links work.
  • I added aliases (i.e., redirects) for all filename changes.
  • If making css changes, I rebuilt the bundle.

@pulumi-bot
Copy link
Collaborator

Your site preview for commit d0925d5 is ready! 🎉

http://pulumi-hugo-origin-pr-3506-d0925d54.s3-website.us-west-2.amazonaws.com.

cnunciato
cnunciato approved these changes Oct 16, 2023
View reviewed changes
Copy link
Contributor

@cnunciato cnunciato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @komalali! 🙌

@komalali komalali merged commit faace17 into master Oct 16, 2023
1 check passed
@komalali komalali deleted the komal/fix-esc-api-routes branch October 16, 2023 19:18
@github-actions
Copy link

Site previews for this pull request have been removed. ✨

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cnunciato cnunciato cnunciato approved these changes

@interurban interurban Awaiting requested review from interurban interurban is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@komalali @pulumi-bot @cnunciato